Skip to content

docs: add DPS310 availability note to MSR-2, MTR-1, and AIR-1#731

Merged
bharvey88 merged 1 commit intomainfrom
docs/dps310-availability-note
Mar 18, 2026
Merged

docs: add DPS310 availability note to MSR-2, MTR-1, and AIR-1#731
bharvey88 merged 1 commit intomainfrom
docs/dps310-availability-note

Conversation

@bharvey88
Copy link
Copy Markdown
Contributor

@bharvey88 bharvey88 commented Mar 18, 2026

Summary

  • Added an inline note to the DPS310 Pressure entry on MSR-2 and MTR-1 sensor definition pages
  • Added an inline note to the DPS310 Pressure Offset entry on AIR-1 sensor definitions page
  • Informs users that a small subset of devices do not include a DPS310 sensor, so they know to expect missing readings if affected

Test plan

  • Verify the note appears correctly on all three sensor definition pages in the rendered docs
  • Confirm no formatting issues in the collapsible sensor sections (MSR-2/MTR-1)

Summary by CodeRabbit

  • Documentation
    • Updated sensor documentation to clarify that DPS310 pressure sensors may not be present on all devices. If sensor readings are absent, this may indicate the device does not include the DPS310 sensor.

…definitions

A small subset of devices do not include a DPS310 sensor. Added an inline note to the DPS310 entries on each product's sensor definitions page so users know to expect missing readings on affected units.
@bharvey88 bharvey88 merged commit a7edfec into main Mar 18, 2026
1 check was pending
@bharvey88 bharvey88 deleted the docs/dps310-availability-note branch March 18, 2026 19:39
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 18, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 099712f1-1352-4c39-ada2-91748c4e3190

📥 Commits

Reviewing files that changed from the base of the PR and between 1f61bbc and 605d864.

📒 Files selected for processing (3)
  • docs/products/air1/setup/sensor-definitions.md
  • docs/products/msr2/setup/msr2-sensor-definitions.md
  • docs/products/mtr1/setup/sensor-definitions.md

Walkthrough

Documentation updates across three sensor definition files clarifying that DPS310 pressure sensors may not be present on some device variants, with notes added to existing sensor descriptions.

Changes

Cohort / File(s) Summary
DPS310 Sensor Documentation Notes
docs/products/air1/setup/sensor-definitions.md, docs/products/msr2/setup/msr2-sensor-definitions.md, docs/products/mtr1/setup/sensor-definitions.md
Added clarifications that DPS310 pressure sensors may be absent on certain device variants, noting the sensor reading or option may not appear if the hardware is not present.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Hop along the sensor paths so clear,
A note for each DPS310 dear,
Some devices skip this pressure friend,
Now docs make sense from start to end!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/dps310-availability-note
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant